Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix for bad gateway error due to missing details on drupal - displays an error message #577

Conversation

OriHoch
Copy link
Contributor

@OriHoch OriHoch commented Jan 15, 2018

see #549 for reproduction steps - although it's hard to reproduce this bug (you need access to user profile which has the required conditions for this bug..)

var tickets_array = [];
var drupal_user_id = _.get(drupal_user, 'user.uid');
if (!drupal_user_id) {
done(false, i18next.t('invalid_user_id'));
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the only change right, all the rest is indentation?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

mostly yes, you can see only the changes without whitespace changes by adding ?w=1 to the url - https://github.com/Midburn/spark/pull/577/files?w=1

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! it's useful.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants